Skip to content

Conversation

@javadzarezadeh
Copy link

I've mapped the Docker container's Alembic folder to the local directory, ensuring that any auto-generated migration files created by Alembic are automatically copied to the local development environment.

@YuriiMotov YuriiMotov added the bug Something isn't working label Sep 4, 2025
@YuriiMotov YuriiMotov changed the title Fix alembic autogenerate issue 🐛 Mount /backend/app/alembic/versions as a volume inside container during development Sep 4, 2025
YuriiMotov

This comment was marked as outdated.

@YuriiMotov YuriiMotov dismissed their stale review September 4, 2025 13:40

Found problem...

YuriiMotov

This comment was marked as resolved.

@YuriiMotov

This comment was marked as resolved.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 5, 2025

As this PR has been waiting for the original user for a while but seems to be inactive, it's now going to be closed. But if there's anyone interested, feel free to create a new PR.

@github-actions github-actions bot closed this Oct 5, 2025
@YuriiMotov YuriiMotov reopened this Oct 5, 2025
@YuriiMotov

This comment was marked as resolved.

@github-actions github-actions bot removed the waiting label Oct 5, 2025
Copy link
Member

@YuriiMotov YuriiMotov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Now changes made in backend/alembic/versions are visible inside the container.

Adding app/alembic/versions to ignore paths is needed to prevent infinite loop

target: /app
ignore:
- ./backend/.venv
- app/alembic/versions
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

./backend/.venv path was invalid - all paths in ignore are relative to path: ./backend. So, it expanded to ./backend/backend/.venv which does not exist.

I removed that invalid ./backend/.venv and added app/alembic/versions to prevent infinite loop on updating files in alembic/versions mounted volume

@YuriiMotov YuriiMotov added feature New feature or request and removed bug Something isn't working labels Oct 10, 2025
@YuriiMotov YuriiMotov changed the title 🐛 Mount /backend/app/alembic/versions as a volume inside container during development ✨ Mount /backend/app/alembic/versions as a volume inside container during development Oct 10, 2025
@github-actions
Copy link
Contributor

This pull request has a merge conflict that needs to be resolved.

@github-actions github-actions bot added the conflicts Automatically generated when a PR has a merge conflict label Jan 20, 2026
@tiangolo
Copy link
Member

Thanks for the interest! But as the user inside the container is different than the user outside, the generated files would belong to a different user ID, making them non-editable by the regular user (unless using sudo), so I prefer to avoid that.

It was like this at some point and I removed it to avoid this problem precisely. So I'll pass on this, but thanks! ☕

@tiangolo tiangolo closed this Jan 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conflicts Automatically generated when a PR has a merge conflict feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants